Skip to content

bpo-35529: Fix a reference counting bug in PyCFuncPtr_FromDll()#11229

Merged
serhiy-storchaka merged 2 commits into
python:masterfrom
ZackerySpytz:bpo-35529-ctypes-ref
Dec 20, 2018
Merged

bpo-35529: Fix a reference counting bug in PyCFuncPtr_FromDll()#11229
serhiy-storchaka merged 2 commits into
python:masterfrom
ZackerySpytz:bpo-35529-ctypes-ref

Conversation

@ZackerySpytz
Copy link
Copy Markdown
Contributor

@ZackerySpytz ZackerySpytz commented Dec 18, 2018

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().

https://bugs.python.org/issue35529

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().
Comment thread Modules/_ctypes/_ctypes.c
@serhiy-storchaka serhiy-storchaka added type-bug An unexpected behavior, bug, or error needs backport to 2.7 skip news labels Dec 20, 2018
@serhiy-storchaka serhiy-storchaka merged commit d77d97c into python:master Dec 20, 2018
@miss-islington
Copy link
Copy Markdown
Contributor

Thanks @ZackerySpytz for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 20, 2018
…honGH-11229)

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().
(cherry picked from commit d77d97c)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
@bedevere-bot
Copy link
Copy Markdown

GH-11260 is a backport of this pull request to the 3.7 branch.

@bedevere-bot
Copy link
Copy Markdown

GH-11261 is a backport of this pull request to the 2.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 20, 2018
…honGH-11229)

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().
(cherry picked from commit d77d97c)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Dec 20, 2018
…11229)

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().
(cherry picked from commit d77d97c)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
miss-islington added a commit that referenced this pull request Dec 20, 2018
…11229)

"dll" would leak if an error occurred in _validate_paramflags() or
GenericPyCData_new().
(cherry picked from commit d77d97c)

Co-authored-by: Zackery Spytz <zspytz@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants